Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixes #36720 - Composable Expandable Table for Smart Proxy content #10716

Closed
wants to merge 1 commit into from

Conversation

sjha4
Copy link
Member

@sjha4 sjha4 commented Aug 31, 2023

What are the changes introduced in this pull request?

Refactor the smart proxy content table as expandable table to prep for more changes on the page in follow-up PRs.

Considerations taken when implementing this change?

What are the testing steps for this pull request?

@sjha4 sjha4 force-pushed the expandable_table_capsule branch from 2fa8027 to a8ddab4 Compare September 1, 2023 17:30
@sjha4 sjha4 force-pushed the expandable_table_capsule branch from a8ddab4 to e82eb5c Compare September 5, 2023 16:49
@sjha4 sjha4 force-pushed the expandable_table_capsule branch from e82eb5c to 60f6bea Compare September 5, 2023 18:17
@sjha4 sjha4 marked this pull request as ready for review September 5, 2023 18:17
@sjha4 sjha4 changed the title Draft Composable Expandable Table for Smart Proxy content Fixes #36720 - Composable Expandable Table for Smart Proxy content Sep 5, 2023
@jeremylenz
Copy link
Member

I think this is the one where I was complaining about the expansion carat, right? I addressed something similar in https://github.com/Katello/katello/pull/10697/files

@sjha4
Copy link
Member Author

sjha4 commented Oct 9, 2023

Seeing your comment now..Will address this in a PR I am working on..Closing this one out.

@sjha4 sjha4 closed this Oct 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants